feat: support application version release documents [PYSDK-122] by arne-aignx · Pull Request #612 · aignostics/python-sdk

arne-aignx · 2026-04-29T13:13:13Z

Summary

Regenerate aignx.codegen from OpenAPI v1.5.0, which adds the four
/api/v1/applications/{id}/versions/{ver}/documents endpoints (list, get
metadata, /file redirect, /content redirect) plus the supporting
VersionDocumentResponse model and VersionDocumentVisibility enum.
Add a new Documents resource on platform (list, details,
download_to_path, read_content)
Add an aignostics application version document CLI subgroup with
list, describe, and download commands.
Update SPEC-APPLICATION-SERVICE, SPEC_PLATFORM_SERVICE, and
SWR-APPLICATION-1-3 to reflect the new behaviour.

Linked items

Jira CR: PYSDK-122
Upstream platform ticket: PAPI-4839
New test case: tests/aignostics/application/TC-APPLICATION-CLI-05.feature

Test plan

make lint passes (ruff + mypy + pyright)
make test_unit passes (new unit tests for Documents resource +
CLI commands incl. happy-path, 404, cache hit, nocache=True)
make audit passes
CI: lint, test matrix (3.11–3.14), CodeQL, Ketryx
Manual smoke: aignostics application version document list <id>
against staging
Manual smoke: aignostics application version document download <id> <name> writes the file using the requested document name

🤖 Generated with Claude Code

Configures the project for the sdlc-* skills: JIRA project key (PYSDK), Atlassian Cloud ID, Ketryx project (Python SDK), per-item-type locations with Git preferred for SWRs and SPECs, and the approval structure pulled from Ketryx project settings. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…PYSDK-122] Regenerate the auto-generated Python client from the public OpenAPI spec v1.5.0, which adds the four release-documents endpoints under /api/v1/applications/{application-id}/versions/{version-id}/documents (list, get metadata, /file, /content) plus the supporting VersionDocumentResponse model and VersionDocumentVisibility enum. This commit is intentionally isolated to keep the generated diff reviewable; the platform-resource wrapper, CLI, and tests follow in subsequent commits. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…e documents [PYSDK-122] Expose the four /api/v1/applications/{id}/versions/{version}/documents endpoints (list, get metadata, /file, /content) added in OpenAPI v1.5.0 through a new Documents resource on the platform module. * New Documents class with list(), details(), download_to_path(), get_download_url(), and get_content_url() methods. The two redirect endpoints (/file and /content) are resolved with allow_redirects=False using the same pattern as Artifact.get_download_url() so the short-lived presigned GCS URL can be inspected before the body is consumed. * New ApplicationVersionDocument Pydantic wrapper around the codegen VersionDocumentResponse model. * Versions.documents(application_id, application_version) returns a bound Documents instance. * list() and details() are cached with the existing application_version_cache_ttl (5 min); the redirect endpoints are intentionally not cached because the URLs are short-lived. * No CRC32C checksum verification on download — PAPI does not expose a checksum for documents; integrity is bounded by HTTPS and signed-URL TTL. * Documents and ApplicationVersionDocument re-exported from the platform package. Unit tests cover happy-path list/details/download/content-url, 404 → NotFoundException, cache hit + nocache=True bypass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…commands [PYSDK-122] Wire the new platform Documents resource into the application CLI so users can discover and download an application version's public release documents without writing Python. * New `aignostics application version` Typer subgroup (mirrors `run`). * Nested `aignostics application version document` subgroup with: * `list APPLICATION_VERSION_ID` — tabulated metadata (id, name, mime type, timestamps), supports `--format text|json`. * `describe APPLICATION_VERSION_ID DOCUMENT_NAME` — full metadata for one document, supports `--format text|json`. Maps NotFoundException to exit code 2 with `Document '{name}' not found for application version '{ver}'.` * `download APPLICATION_VERSION_ID DOCUMENT_NAME [--output PATH]` — streams the file to disk via the platform 307 redirect; default destination is the current working directory using the document's server-provided filename. APPLICATION_VERSION_ID accepts either `application_id` (latest version is resolved via Versions.latest) or `application_id:version_number` for an explicit semantic version pin. Unit tests cover the four BDD scenarios from TC-APPLICATION-CLI-05: list, describe, describe-not-found, and download. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…en [PYSDK-122] * Bump Date / Version frontmatter on SPEC-APPLICATION-SERVICE and SPEC_PLATFORM_SERVICE to today's date. * Fix lint findings surfaced by `make lint` after the codegen + Documents resource changes (DOC201 / DOC501 / PLC1901 / PLR6104 / formatting + pyright reportCallIssue on cached_operation's injected `nocache` kwarg). * Add the new required `input_artifacts` field that the v1.5.0 ItemResultReadResponse model now demands to the test helper in tests/aignostics/application/utils_test.py — without it three utils_test cases fail Pydantic validation. `make lint`, `make test_unit`, and `make audit` all pass after this change. `make test_integration` reports one pre-existing GUI-pagination failure that hits the production API without a JWT and was already failing on the parent commit (no relation to this CR). The CHANGELOG.md is generated by git-cliff at release time; the per-CR entry will be produced from the conventional-commit messages in this branch when `make publish-release` runs, so no manual entry is added. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…t download Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot

Pull request overview

Adds SDK + CLI support for application version release documents by updating the generated OpenAPI client to v1.5.0 and introducing a new Documents resource reachable via client.applications.versions.documents(application_id, version).

Changes:

Regenerates aignx.codegen from OpenAPI 1.5.0, including version document endpoints/models and input_artifacts in item results.
Adds Documents + ApplicationVersionDocument to the platform resources, including redirect-resolution and download helpers.
Adds aignostics application version document CLI subgroup and updates specs/requirements + unit/feature tests.

Reviewed changes

Copilot reviewed 12 out of 59 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
tests/aignostics/platform/resources/applications_test.py	Adds unit coverage for the new `Documents` resource (list/details/redirect/download + cache behavior).
tests/aignostics/application/utils_test.py	Updates test helper to populate `input_artifacts` to match updated codegen models.
tests/aignostics/application/cli_test.py	Adds CLI unit tests for `application version document` commands (list/describe/download + 404 handling).
tests/aignostics/application/TC-APPLICATION-CLI-05.feature	Introduces BDD test case text for version document CLI behavior.
src/aignostics/platform/resources/applications.py	Implements `Versions.documents()`, `Documents` resource, and `ApplicationVersionDocument` wrapper model.
src/aignostics/platform/init.py	Exposes `Documents` and `ApplicationVersionDocument` in the public platform API surface.
src/aignostics/application/_cli.py	Adds `application version document` CLI subgroup + parsing/resolution helpers.
specifications/SPEC_PLATFORM_SERVICE.md	Updates platform spec to include release document support + schema and API snippets.
specifications/SPEC-APPLICATION-SERVICE.md	Updates application spec to include CLI capabilities for release documents.
requirements/SWR-APPLICATION-1-3.md	Adds new functional requirement for listing/describing/downloading release documents.
codegen/out/docs/PublicApi.md	Documents the newly generated public endpoints for version documents.
codegen/out/aignx/codegen/rest.py	Bumps generated OpenAPI doc version marker to 1.5.0.
codegen/out/aignx/codegen/models/version_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/version_document_visibility.py	Adds new generated enum model for document visibility.
codegen/out/aignx/codegen/models/version_document_response.py	Adds new generated model for version document metadata.
codegen/out/aignx/codegen/models/validation_error_loc_inner.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/validation_error.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/user_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/scheduling_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/scheduling_request.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_termination_reason.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_state.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_output.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_item_statistics.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_creation_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/run_creation_request.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/output_artifact_visibility.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/output_artifact_scope.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/output_artifact_result_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/output_artifact.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/organization_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/me_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/item_termination_reason.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/item_state.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/item_result_read_response.py	Adds `input_artifacts` field + serialization/deserialization updates.
codegen/out/aignx/codegen/models/item_output.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/item_creation_request.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/input_artifact_result_read_response.py	Adds new generated model for input artifact results.
codegen/out/aignx/codegen/models/input_artifact_creation_request.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/input_artifact.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/http_validation_error.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/custom_metadata_update_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/custom_metadata_update_request.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/artifact_termination_reason.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/artifact_state.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/artifact_output.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/application_version.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/application_read_short_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/application_read_response.py	Updates generated header/version marker.
codegen/out/aignx/codegen/models/init.py	Exports newly generated models (`VersionDocument*`, `InputArtifactResultReadResponse`).
codegen/out/aignx/codegen/exceptions.py	Updates generated header/version marker.
codegen/out/aignx/codegen/configuration.py	Updates generated header/version marker + debug report API version string.
codegen/out/aignx/codegen/api_client.py	Updates generated header/version marker.
codegen/out/.openapi-generator/FILES	Updates generator file manifest to include new generated models.
codegen/in/openapi.json	Updates input OpenAPI spec to 1.5.0 and adds the new version document endpoints/schemas.
codegen/in/archive/openapi_1.5.0.json	Adds archived copy of the 1.5.0 OpenAPI spec.
CLAUDE.md	Adds SDLC configuration section (process/documentation metadata).

…[PYSDK-122] Drop the get_content_url / get_download_url public methods on the Documents resource and the _resolve_redirect_url / _fetch_redirect_url machinery that intercepted the platform 307 to extract a signed URL — there were no in-SDK callers for the URL surface, and following the redirect with requests (allow_redirects=True) is sufficient for download_to_path. requests strips the Authorization header on the cross-host hop, so the bearer token is not forwarded to GCS. Also address Copilot review on PR #612: - URL-encode each path segment in download_to_path (application_id, version, document_name) so reserved characters in a document name cannot inject extra path segments. - Split application/version "is unavailable" 404s from document-level 404s in the describe / download CLI commands so error messages stop conflating the two failure modes. - Update SPEC_PLATFORM_SERVICE.md so the resource table and Documents class snippet match the implementation (constructor signature, list() return type, no get_content_url). - Reword TC-APPLICATION-CLI-05 scenario 04 to match actual behaviour: the document file is written using the requested document name, not a server-provided filename. Net change: -167 lines after redirect simplification; +13 / -25 across the remaining files for the encoding and CLI split. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

codecov · 2026-04-29T14:03:43Z

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
859	1	858	15

View the top 1 failed test(s) by shortest run time

tests.aignostics.platform.resources.applications_test::test_documents_download_to_path_rejects_non_directory

Stack Traces | 0.006s run time

documents = <aignostics.platform.resources.applications.Documents object at 0x7fa2e3d2fdf0>
tmp_path = PosixPath('.../pytest-3/popen-gw1/test_documents_download_to_pat2')

    @pytest.mark.unit
    def test_documents_download_to_path_rejects_non_directory(documents: Documents, tmp_path: Path) -> None:
        """download_to_path() raises ValueError when destination is not an existing directory."""
        file_path = tmp_path / "some_file.pdf"
        file_path.write_bytes(b"content")
    
        with pytest.raises(ValueError, match="must be a directory"):
>           documents.download_to_path(DOCUMENT_OUTPUT_DESCRIPTION_PDF, file_path)

.../platform/resources/applications_test.py:348: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
.../platform/resources/applications.py:450: in download_to_path
    destination_path = self._resolve_destination_path(destination, document_name)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

destination = PosixPath('.../pytest-3/popen-gw1/test_documents_download_to_pat2/some_file.pdf')
document_name = 'output_description.pdf'

    @staticmethod
    def _resolve_destination_path(destination: Path | str, document_name: str) -> Path:
        """Resolve the on-disk path to write a document to and ensure its parent exists.
    
        Returns:
            Path: The absolute, parent-created destination path.
    
        Raises:
            ValueError: If the destination is an existing file or a non-existent path
                with an existing parent that is a file.
        """
        destination_path = Path(destination)
        if destination_path.is_file() or (destination_path.exists() and not destination_path.is_dir()):
            msg = f"Destination '{destination}' is an existing file. Please provide a directory or a non-existent path."
>           raise ValueError(msg)
E           ValueError: Destination '.../pytest-3/popen-gw1/test_documents_download_to_pat2/some_file.pdf' is an existing file. Please provide a directory or a non-existent path.

.../platform/resources/applications.py:492: ValueError

During handling of the above exception, another exception occurred:

documents = <aignostics.platform.resources.applications.Documents object at 0x7fa2e3d2fdf0>
tmp_path = PosixPath('.../pytest-3/popen-gw1/test_documents_download_to_pat2')

    @pytest.mark.unit
    def test_documents_download_to_path_rejects_non_directory(documents: Documents, tmp_path: Path) -> None:
        """download_to_path() raises ValueError when destination is not an existing directory."""
        file_path = tmp_path / "some_file.pdf"
        file_path.write_bytes(b"content")
    
>       with pytest.raises(ValueError, match="must be a directory"):
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
E       AssertionError: Regex pattern did not match.
E         Expected regex: 'must be a directory'
E         Actual message: "Destination '.../pytest-3/popen-gw1/test_documents_download_to_pat2/some_file.pdf' is an existing file. Please provide a directory or a non-existent path."

.../platform/resources/applications_test.py:347: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

…K-122] Split Documents.download_to_path into three helpers (_resolve_destination_path, _build_document_endpoint_url, _stream_document) so cognitive complexity drops below the S3776 threshold. Lift recurring test literals into module-level constants where it improves clarity, and apply NOSONAR with rule + reason for literals where a constant would be less informative than the literal itself. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot

Pull request overview

Copilot reviewed 12 out of 59 changed files in this pull request and generated 4 comments.

…YSDK-122] Adds the in-memory read variant of release-document download backed by GET .../documents/{name}/content. Unlike /file (which carries a Content-Disposition: attachment header for browser downloads), /content lets the GCS-stored Content-Type pass through and is intended for programmatic consumers that want the raw bytes (small JSON manifests, license text, etc.). Refactors _build_document_endpoint_url to take a suffix parameter and _stream_document to take a writer callable instead of opening the file itself, so download_to_path and read_content share the request, retry, and error-mapping path. Updates the Documents SPEC to advertise the new method. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…SDK-122] Adds 15 unit tests for the document list/describe/download CLI commands covering the unresolvable-version path, the missing-document path, and generic-failure path in both text and JSON output formats. Reuses test literals via shared module-level constants to keep SonarCloud's S1192 duplicate-string check quiet. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot

Pull request overview

Copilot reviewed 12 out of 59 changed files in this pull request and generated 2 comments.

…K-122] Replace non-standard `# noqa: ... -- justification` form with the documented ruff syntax plus a preceding rationale comment, and lift the duplicated "Not Found" / "a.pdf" string literals into module-level test constants so the duplicate-literal smell is resolved without NOSONAR suppressions. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

…R [PYSDK-122]

olivermeyer · 2026-04-30T11:26:19Z

+- `version document list APPLICATION_VERSION_ID`: List public release documents attached to an application version
+- `version document describe APPLICATION_VERSION_ID DOCUMENT_NAME`: Show metadata for a single public release document
+- `version document download APPLICATION_VERSION_ID DOCUMENT_NAME [--output PATH]`: Download a public release document to a local path


Suggested change

- `version document list APPLICATION_VERSION_ID`: List public release documents attached to an application version

- `version document describe APPLICATION_VERSION_ID DOCUMENT_NAME`: Show metadata for a single public release document

- `version document download APPLICATION_VERSION_ID DOCUMENT_NAME [--output PATH]`: Download a public release document to a local path

- `version document list`: List public release documents attached to an application version

- `version document describe`: Show metadata for a single public release document

- `version document download`: Download a public release document to a local path

We don't document the arguments to the other commands. I would suggest doing the same here for consistency.

The style was a bit unclear to me.
- run list [--for-organization ORG_ID]: List application runs; supports listing all runs for an organization with --for-organization (only available to org admins)
documents the options.
Fine to remove for me.

Oops.

Since we're already not consistent, I don't really mind either way. We can leave it as is.

olivermeyer · 2026-04-30T11:37:36Z

+
+@document_app.command("list")
+def application_version_document_list(
+    application_version_id: Annotated[str, typer.Argument(..., help=_APPLICATION_VERSION_ID_HELP)],


Other commands take the application ID and version as separate arguments (example). Is there a reason for doing it differently here, or should we do the same for consistency?

Oh, I will change that. Was mislead by claude who was adamant, that this is the way to go in PySDK

olivermeyer · 2026-04-30T11:40:44Z

+    if version_number is None:
+        # Resolve the latest version for the application.
+        version = client.applications.versions.latest(application=application_id)
+        if version is None:
+            raise NotFoundException(
+                status=404,
+                reason=f"No versions found for application '{application_id}'.",
+            )
+        version_number = version.number


This is pretty much the same as this, maybe we can do client.application_version(...)?

I'll refactor. A bit unfamiliar with the code base here.

olivermeyer · 2026-04-30T11:50:23Z

+Layer: System (backend logic)
+---
+
+System shall list, describe, and download release documents attached to a given application version, exposing only documents with public visibility and uploaded status.


exposing only documents with public visibility and uploaded status.

This suggests the Python SDK pulls all documents, including internal docs, and only exposes public docs. In reality the API only exposes public docs, right? So Python SDK just exposes whatever it receives.

What is PythonSDKs stance on surfacing fields?
IMO we could also get rid of the visibility all-together.
The API does expose it, but as you said correctly. PythonSDK will never see the internal documents.

Yes, I would remove the notion of visibility: Python SDK shows the user whatever the platform exposes.

olivermeyer · 2026-04-30T11:51:00Z

+    Given the user has access to an application version with release documents attached
+    When the user requests the list of release documents for the application version
+    Then the system shall return metadata for documents with public visibility and uploaded status
+    And the system shall exclude documents with internal visibility or pending status


Similar to my comment above; AFAIU Python SDK does not exclude anything. Or am I missing something?

olivermeyer · 2026-04-30T11:52:02Z

+  @id:TC-APPLICATION-CLI-05-03
+  Scenario: System rejects requests for non-existent or non-public release documents
+    Given the user has access to an application version
+    When the user requests metadata for a document that does not exist or is not public


If my understanding is correct, non-existent or non-public is the same for Python SDK.

Copilot

Pull request overview

Copilot reviewed 12 out of 59 changed files in this pull request and generated 7 comments.

Copilot

Pull request overview

Copilot reviewed 12 out of 59 changed files in this pull request and generated 4 comments.

sonarqubecloud · 2026-05-04T08:26:45Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

arne-aignx and others added 7 commits April 29, 2026 14:09

chore: TODO - add commit content

67a8ffb

fix(application): silence pyright unused-variable warnings in documen…

87e49d9

…t download Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings April 29, 2026 13:13

arne-aignx requested review from a team and helmut-hoffer-von-ankershoffen as code owners April 29, 2026 13:13

Copilot started reviewing on behalf of arne-aignx April 29, 2026 13:13 View session

Copilot AI reviewed Apr 29, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings April 29, 2026 14:07

Copilot started reviewing on behalf of arne-aignx April 29, 2026 14:08 View session

Copilot AI reviewed Apr 29, 2026

View reviewed changes

Comment thread src/aignostics/platform/resources/applications.py

Comment thread src/aignostics/platform/resources/applications.py

Comment thread src/aignostics/platform/resources/applications.py Outdated

Comment thread specifications/SPEC_PLATFORM_SERVICE.md Outdated

arne-aignx and others added 3 commits April 29, 2026 17:19

fix: test after model update

e73ecdd

Copilot AI review requested due to automatic review settings April 30, 2026 06:01

Copilot started reviewing on behalf of arne-aignx April 30, 2026 06:02 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Comment thread src/aignostics/platform/resources/applications.py

Comment thread src/aignostics/platform/resources/applications.py

arne-aignx and others added 2 commits April 30, 2026 09:21

fixup! refactor(platform): address SonarCloud findings on documents P…

a7761ca

…R [PYSDK-122]

olivermeyer reviewed Apr 30, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings April 30, 2026 14:05

Copilot started reviewing on behalf of arne-aignx April 30, 2026 14:06 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

arne-aignx force-pushed the feat/PAPI-4839-add-support-for-release-documents-api branch from 7719458 to a0028a7 Compare April 30, 2026 14:56

chore: address review comments

4cac6f1

Copilot AI review requested due to automatic review settings May 4, 2026 07:53

arne-aignx force-pushed the feat/PAPI-4839-add-support-for-release-documents-api branch from a0028a7 to 4cac6f1 Compare May 4, 2026 07:53

Copilot started reviewing on behalf of arne-aignx May 4, 2026 07:54 View session

Copilot AI reviewed May 4, 2026

View reviewed changes

Comment thread src/aignostics/application/_cli.py

Comment thread src/aignostics/platform/resources/applications.py Outdated

Comment thread src/aignostics/platform/resources/applications.py

Comment thread src/aignostics/application/_cli.py

Remove option to set file name for downloading release documents

99ecbe8

Conversation

arne-aignx commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Linked items

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov Bot commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ 1 Tests Failed:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

arne-aignx commented Apr 29, 2026 •

edited

Loading

codecov Bot commented Apr 29, 2026 •

edited

Loading